Skip to content

Conversation

@OriolMunoz-da
Copy link
Contributor

Part of #3562

I expect things to break (missing permissions). After this is deployed:

  • Confirm that the env vars are readable within a GHA job
  • Add the job to run the performance tests

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
enabled: true,
maintenanceWindow: { day: 2, hour: 8 },
protected: false,
tier: 'db-custom-2-7680', // same as devnet & testnet as of Jan 2026
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the actual configuration is available in the internal repo, not in this one...
that being said, it's likely we'll increase the DB size, not make it smaller, so this should be fine as baseline

HELM_MAX_HISTORY_SIZE: 42,
imagePullSecretByNamespaceNameForServiceAccount: () => [],
infraAffinityAndTolerations: {},
CloudPostgres: function CloudPostgres() {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test below breaks without this

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
name: GHA_NAMESPACE_NAME,
},
});
const exactNs: ExactNamespace = {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One day I'll understand why we have this ExactNamespace all over the place...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you mean this?

// There is no way to read the logical name off a Namespace. Exactly

Copy link
Contributor

@isegall-da isegall-da left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you

@OriolMunoz-da OriolMunoz-da merged commit 35543de into main Jan 21, 2026
62 checks passed
@OriolMunoz-da OriolMunoz-da deleted the oriol/ci-perftests-cloudsql branch January 21, 2026 14:13
OriolMunoz-da added a commit that referenced this pull request Jan 21, 2026
OriolMunoz-da added a commit that referenced this pull request Jan 21, 2026
This reverts commit 35543de.

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
OriolMunoz-da added a commit that referenced this pull request Jan 21, 2026
* Revert "[static] Add missing secretKeyRef in GHA setup (#3638)"

This reverts commit 080c7d9.

* Revert "Deploy CloudSQL instance for performance tests (#3634)"

This reverts commit 35543de.

* [ci] run

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>

---------

Signed-off-by: Oriol Muñoz <oriol.munoz@digitalasset.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants